#2273: add Git as global tool by JoelAdbu · Pull Request #2721 · devonfw/IDEasy

JoelAdbu · 2026-10-21T12:49:53Z

This PR fixes #2273

This PR continues the work of #2360.

Implemented changes:

Added Git as a global IDEasy tool and registered the Git commandlet.
Added Linux installation support through native package managers (apt and zypper).
Added Macos installation support through native package manager (brew).
Added Windows installation support using the Git for Windows installer with silent installer arguments.
Added Windows registry-based detection and uninstall support for Git.
Improved Windows registry matching by validating the application's DisplayName.
Added generic installer argument support for global tools.
Added support for global tools that do not require repository version resolution when installed through the native Linux package manager.
Updated the CHANGELOG.

Testing instructions:

Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:

Build IDEasy and make the local development version available with build-local-dev.sh
For all 3 Linux, Windows and Macos run: ide install git and / or ide uninstall git

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

When running mvn clean test locally all tests pass and build is successful
PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat and not feature/921 fixed setup.bat). If no issue ID exists, title only.
PR top-level comment summaries what has been done and contains link to addressed issue(s)
PR and issue(s) have suitable labels
Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
You followed all coding conventions
You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labelled
with internal
You have not changed any dependency in pom.xml files or otherwise if runtime dependencies changed, you have updated our LICENSE.asciidoc
You have formulated clear instructions on how to test your contribution under "Testing instructions"

Checklist for tool commandlets

Have you added a new «tool» as commandlet? There are the following additional checks:

The tool can be installed automatically (during setup via settings) or via the commandlet call
The tool is isolated in its IDEasy project, see Sandbox Principle
The new tool is added to the table of tools in LICENSE.asciidoc
The new commandlet is a command-wrapper for «tool»
Proper help texts for all supported languages are added here
The new commandlet installs potential dependencies automatically
The variables «TOOL»_VERSION and «TOOL»_EDITION are honored by your commandlet
The new commandlet is tested on all platforms it is available for or tested on all platforms that are in scope of the linked issue

…ate-git-as-global-tool # Conflicts: # CHANGELOG.adoc

coveralls · 2026-10-21T12:55:28Z

Coverage Report for CI Build 40846440246

Coverage increased (+0.08%) to 75.319%

Details

Coverage increased (+0.08%) from the base build.
Patch coverage: No coverable lines changed in this PR.
306 coverage regressions across 7 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

306 previously-covered lines in 7 files lost coverage.

File	Lines Losing Coverage	Coverage
com/devonfw/tools/ide/context/AbstractIdeContext.java	131	72.08%
com/devonfw/tools/ide/git/GitContextImpl.java	51	41.82%
com/devonfw/tools/ide/tool/IdeasyCommandlet.java	51	75.55%
com/devonfw/tools/ide/os/WindowsHelperImpl.java	48	56.36%
com/devonfw/tools/ide/tool/GlobalToolCommandlet.java	15	60.79%
com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java	8	91.91%
com/devonfw/tools/ide/version/VersionSegment.java	2	90.29%

Coverage Stats


Relevant Lines:	19899
Covered Lines:	15644
Line Coverage:	78.62%
Relevant Branches:	8856
Covered Branches:	6014
Branch Coverage:	67.91%
Branches in Coverage %:	Yes
Coverage Strength:	3.38 hits per line

💛 - Coveralls

- small improvement in log messages

- added support for git on mac-os

- improvements

…it-as-global-tool # Conflicts: # CHANGELOG.adoc # cli/src/main/java/com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java # cli/src/main/java/com/devonfw/tools/ide/tool/GlobalToolCommandlet.java

- added tests for git

quando632

Nice continuation of #2360. The isPackageManagerBasedInstallation() and getInstallerArguments() hooks keep Git's OS-specific behaviour in Git instead of the shared base class, and the DisplayName check in WindowsHelperImpl fixes a real mismatch, since a registry search for "Git" also hits keys like GitHub CLI. The new Windows registry tests and the no-Git fallback tests in GitOperationTest cover the tricky branches well.

quando632 · 2026-10-27T13:55:45Z

+    } else if ((urlsPath != null) && Files.isDirectory(urlsPath)) {
+      LOG.debug("Git is not available. Using existing URL metadata without updating it.");
+    } else {
+      gitContext.findGitRequired();


On a machine without Git and without _ide/urls (a fresh Windows setup, the use case of #2273), ide install git resolves its version via getUrls() and lands here. findGitRequired() then aborts with the new hint "Try running ide install git" from GitContextImpl.java:351, so the user is sent in a circle. Please either keep the download hint without the ide install git advice when the urls repository is missing, or provide a way to get the Git metadata without Git.

quando632 · 2026-10-27T13:55:45Z

    return DEFAULT_WINDOWS_GIT_PATH;
  }

+  private boolean isInstallingGit(AbstractCommandlet cmd) {


isInstallingGit puts tool-specific knowledge into the generic context and reads InstallCommandlet.tool directly. A generic guard such as getGitContext().findGit() != null around the settings-update check would cover ide install git the same way, without coupling AbstractIdeContext to Git, and would also avoid the same failure for other commands run without Git.

quando632 · 2026-10-27T13:55:45Z

    return null;
  }
+
+  protected List<String> getInstallerArguments() {


getInstallerArguments() is a new protected extension point but has no Javadoc, unlike its neighbours such as getNativePackages().

Suggested change

protected List<String> getInstallerArguments() {

/**

* @return the additional arguments passed to the downloaded installer executable (e.g. silent mode flags). The default is an empty {@link List}.

*/

protected List<String> getInstallerArguments() {

…it-as-global-tool # Conflicts: # CHANGELOG.adoc # cli/src/main/java/com/devonfw/tools/ide/tool/GlobalToolCommandlet.java

- review requests

cap-juan and others added 3 commits October 5, 2026 11:26

devonfw#2273: add Git as global tool

10dd94d

devonfw#2273: solve merge conflicts

ca1d925

Merge remote-tracking branch 'upstream/main' into feature/2273-integr…

e9da9d1

…ate-git-as-global-tool # Conflicts: # CHANGELOG.adoc

github-project-automation Bot added this to IDEasy board Oct 21, 2026

github-project-automation Bot moved this to 🆕 New in IDEasy board Oct 21, 2026

github-actions Bot assigned JoelAdbu Oct 21, 2026

JoelAdbu mentioned this pull request Oct 21, 2026

#2273: add Git as global tool #2360

Closed

17 tasks done

JoelAdbu added enhancement New feature or request commandlet ide sub-command install installation process of IDE + tools and install commandlet git git version management tool integration labels Oct 21, 2026

devonfw#2273:

7e3757b

- small improvement in log messages

JoelAdbu moved this from 🆕 New to Team Review in IDEasy board Oct 21, 2026

JoelAdbu and others added 5 commits October 22, 2026 11:06

devonfw#2273:

ff14d3b

- added support for git on mac-os

devonfw#2273:

9e661b9

- improvements

Merge remote-tracking branch 'upstream' into feature/2273-integrate-g…

fa02095

…it-as-global-tool # Conflicts: # CHANGELOG.adoc # cli/src/main/java/com/devonfw/tools/ide/commandlet/CommandletManagerImpl.java # cli/src/main/java/com/devonfw/tools/ide/tool/GlobalToolCommandlet.java

devonfw#2273:

14543a6

- added tests for git

Merge branch 'main' into feature/2273-integrate-git-as-global-tool

54515d4

quando632 self-assigned this Oct 27, 2026

quando632 requested changes Oct 27, 2026

View reviewed changes

JoelAdbu added 2 commits October 28, 2026 09:25

Merge remote-tracking branch 'upstream' into feature/2273-integrate-g…

ba55884

…it-as-global-tool # Conflicts: # CHANGELOG.adoc # cli/src/main/java/com/devonfw/tools/ide/tool/GlobalToolCommandlet.java

devonfw#2273:

be89d7f

- review requests

Navigation Menu

Repository navigation

#2273: add Git as global tool - #2721

#2273: add Git as global tool#2721
JoelAdbu wants to merge 11 commits into
devonfw:mainfrom
JoelAdbu:feature/2273-integrate-git-as-global-tool

JoelAdbu commented Oct 21, 2026 •

edited

Loading

Uh oh!

coveralls commented Oct 21, 2026 •

edited

Loading

Uh oh!

quando632 left a comment

Uh oh!

quando632 Oct 27, 2026

Uh oh!

Uh oh!

quando632 Oct 27, 2026

Uh oh!

quando632 Oct 27, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

JoelAdbu commented Oct 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

This PR fixes #2273

Implemented changes:

Testing instructions:

Checklist for this PR

Checklist for tool commandlets

Uh oh!

coveralls commented Oct 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for CI Build 40846440246

Coverage increased (+0.08%) to 75.319%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

quando632 left a comment

Choose a reason for hiding this comment

Uh oh!

quando632 Oct 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

quando632 Oct 27, 2026

Choose a reason for hiding this comment

Uh oh!

quando632 Oct 27, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

JoelAdbu commented Oct 21, 2026 •

edited

Loading

coveralls commented Oct 21, 2026 •

edited

Loading